Skip to content

remove usage of deprecated getMockForAbstractClass#7681

Merged
simfeld merged 3 commits intoecamp:develfrom
BacLuc:fix-abstract-mock-deprecated
Jul 22, 2025
Merged

remove usage of deprecated getMockForAbstractClass#7681
simfeld merged 3 commits intoecamp:develfrom
BacLuc:fix-abstract-mock-deprecated

Conversation

@BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Jun 29, 2025

AbstractPersistProcessorTest: remove usage of deprecated getMockForAbstractClass

Replace the functionality we need in the test with injected callbacks.

Issue: #7594


AbstractRemoveProcessorTest: remove usage of deprecated getMockForAbstractClass

Replace the functionality we need in the test with injected callbacks.

Issue: #7594


Types/Doctrine: remove usage of deprecated getMockForAbstractClass

Just instantiate a concrete implementation, we did not verify the interactions.
The Postgres implementation does not return the TimeZone when converting
to DatabaseValue.

Issue: #7594

BacLuc added 3 commits June 29, 2025 22:07
…stractClass

Replace the functionality we need in the test with injected callbacks.

Issue: ecamp#7594
…tractClass

Replace the functionality we need in the test with injected callbacks.

Issue: ecamp#7594
Just instantiate a concrete implementation, we did not verify the interactions.
The Postgres implementation does not return the TimeZone when converting
to DatabaseValue.

Issue: ecamp#7594
@BacLuc BacLuc requested a review from a team June 29, 2025 20:16
@BacLuc BacLuc requested a review from a team June 30, 2025 13:58
@simfeld simfeld added this pull request to the merge queue Jul 22, 2025
Merged via the queue into ecamp:devel with commit f57b089 Jul 22, 2025
30 checks passed
This was referenced Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants